Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

global AT-TLS configuration #3668

Merged
merged 13 commits into from
May 23, 2024
Merged

global AT-TLS configuration #3668

merged 13 commits into from
May 23, 2024

Conversation

achmelo
Copy link
Member

@achmelo achmelo commented May 17, 2024 •
edited
Loading

Describe your pull request here:
Restructure the AT-TLS configuration guide so there is 1 general page that links to component-specific configuration.
List the file(s) included in this PR:

After creating the PR, follow the instructions in the comments.

@github-actions GitHub Actions
Copy link

😺 Thank you for creating this PR! To publish your content to Zowe Docs, follow these required steps.

  • Add the label review: doc.
  • Identify your content topic with a label. (Examples: area: apiml, area: cli, area: install and config, etc.)
  • Specify the major Zowe release(s) for your content. (Examples: release: V1, release: V2, release: V3)
    • If adding content that needs to be removed from V3 documentation, add the V3 N/A tag.
  • Select the master branch if your PR updates content that is on the live site. Select docs-staging if your PR updates content for a future release.
  • Notify the Doc Squad about this PR. If you don't know whom should review your content, message the #zowe-doc Slack channel. If you know which Doc Squad writer should approve your content, add that person as a reviewer.

Need help? Contact the Doc Squad in the #zowe-doc Slack channel.

@github-actions GitHub Actions
Copy link

github-actions bot commented May 17, 2024 •
edited
Loading

📁 The PR description is missing the file name(s) for the updated content. List all the files included in this PR so this information displays in our Zowe Docs GitHub Slack channel.

If you have addressed this issue already, refresh this page in your browser to remove this comment.

@github-actions GitHub Actions
Copy link

💾 The release label is missing. Add a release: label so your content is published with the correct major Zowe release.

If you have addressed this issue already, refresh this page in your browser to remove this comment.

@github-actions GitHub Actions
Copy link

🔍 The review label is missing. Add a review: label so we can determine who needs to approve this PR.

If you have addressed this issue already, refresh this page in your browser to remove this comment.

@github-actions GitHub Actions
Copy link

📌 The subject area label is missing. Add an area: label so we know what your content is about.

If you have addressed this issue already, refresh this page in your browser to remove this comment.

@github-actions GitHub Actions
Copy link

github-actions bot commented May 17, 2024 •
edited
Loading

🚀 Deployed on https://664df2ed605eaa03f743d4f4--zowe-docs-master.netlify.app

@github-actions github-actions bot temporarily deployed to pull request May 17, 2024 13:00 Inactive
@achmelo
fix typo
d240e0d 
Signed-off-by: achmelo <[email protected]>
@github-actions github-actions bot temporarily deployed to pull request May 17, 2024 13:48 Inactive
@balhar-jakub
Copy link
Member

Doesn't this belong to the V3 documentation?

@janan07
minor language refactor
a2d6c4c 
Signed-off-by: Andrew Jandacek <[email protected]>
janan07
janan07 approved these changes May 21, 2024
View reviewed changes
Copy link
Collaborator

@janan07 janan07 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed and edited by doc squad

@github-actions github-actions bot temporarily deployed to pull request May 21, 2024 10:30 Inactive
@achmelo
Copy link
Member Author

achmelo commented May 21, 2024

Doesn't this belong to the V3 documentation?

is there some reason why it should not be in v2 as well?

@github-actions github-actions bot temporarily deployed to pull request May 21, 2024 12:37 Inactive
@balhar-jakub
Copy link
Member

If it works for V2 as well, then there is probably none.

@janan07
minor language refactoring
d7e8a43 
Signed-off-by: Andrew Jandacek <[email protected]>
@janan07
change title
13515ac 
Signed-off-by: Andrew Jandacek <[email protected]>
janan07
janan07 approved these changes May 22, 2024
View reviewed changes
Copy link
Collaborator

@janan07 janan07 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed nd edited by doc squad

@github-actions github-actions bot temporarily deployed to pull request May 22, 2024 13:32 Inactive
@achmelo achmelo merged commit 3b3213e into docs-staging May 23, 2024
4 of 5 checks passed
@achmelo achmelo deleted the reboot/attls-conf branch May 23, 2024 07:32
janan07 added a commit that referenced this pull request May 23, 2024
@janan07
Revert "global AT-TLS configuration (#3668)"
5d163f1 
This reverts commit 3b3213e.
@janan07 janan07 mentioned this pull request May 23, 2024
Merged
achmelo added a commit that referenced this pull request May 27, 2024
@achmelo
Revert "Revert "global AT-TLS configuration (#3668)""
4064a29 
This reverts commit 5d163f1.

Signed-off-by: achmelo <[email protected]>
achmelo added a commit that referenced this pull request May 28, 2024
@achmelo @janan07
Refatctor and update AT-TLS configuration (#3683)
ffd3041 
* configure infinispan with zowe yaml

Signed-off-by: achmelo <[email protected]>

* restructure AT-TLS conf to use global parameters

Signed-off-by: achmelo <[email protected]>

* Revert "Revert "global AT-TLS configuration (#3668)""

This reverts commit 5d163f1.

Signed-off-by: achmelo <[email protected]>

* gateway configuration

Signed-off-by: achmelo <[email protected]>

* minor language typo correction

Signed-off-by: Andrew Jandacek <[email protected]>

---------

Signed-off-by: achmelo <[email protected]>
Signed-off-by: Andrew Jandacek <[email protected]>
Co-authored-by: Andrew Jandacek <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pj892031 pj892031 pj892031 left review comments

@janan07 janan07 janan07 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@achmelo @balhar-jakub @janan07 @pj892031